Skip to content

chore(e2e): add tests for RBAC policyDecisionPrecedence config #3583

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore(e2e): add tests for RBAC policyDecisionPrecedence config #3583

wants to merge 1 commit into from
+103 −0

Conversation

@JessicaJHee
Copy link
Member

@JessicaJHee JessicaJHee commented Oct 17, 2025

Description

Adds tests for the new policyDecisionPrecedence config

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@openshift-ci
Copy link

openshift-ci bot commented Oct 17, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 17, 2025

/test e2e-tests

@qodo-merge-pro
Copy link

qodo-merge-pro bot commented Oct 17, 2025

You are above your monthly Qodo Merge usage quota. For more information, please visit here.

@openshift-ci
Copy link

openshift-ci bot commented Oct 17, 2025

@JessicaJHee: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test e2e-ocp-helm

The following commands are available to trigger optional jobs:

/test cleanup-mapt-destroy-orphaned-aks-clusters

/test cleanup-mapt-destroy-orphaned-eks-clusters

/test e2e-aks-helm-nightly

/test e2e-aks-operator-nightly

/test e2e-eks-helm-nightly

/test e2e-eks-operator-nightly

/test e2e-gke-helm-nightly

/test e2e-gke-operator-nightly

/test e2e-ocp-helm-nightly

/test e2e-ocp-helm-sealights-nightly

/test e2e-ocp-helm-upgrade-nightly

/test e2e-ocp-operator-auth-providers-nightly

/test e2e-ocp-operator-nightly

/test e2e-ocp-v4-17-helm-nightly

/test e2e-ocp-v4-19-helm-nightly

/test e2e-osd-gcp-helm-nightly

/test e2e-osd-gcp-operator-nightly

Use /test all to run the following jobs that were automatically triggered:

pull-ci-redhat-developer-rhdh-main-e2e-ocp-helm

In response to this:

/test e2e-tests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 17, 2025

/test all

@qodo-merge-pro
Copy link

qodo-merge-pro bot commented Oct 17, 2025

You are above your monthly Qodo Merge usage quota. For more information, please visit here.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2025

The image is available at:

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from 8697b1e to 663e3a5 Compare October 20, 2025 19:27
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 20, 2025

/test all

@qodo-merge-pro
Copy link

qodo-merge-pro bot commented Oct 20, 2025

You are above your monthly Qodo Merge usage quota. For more information, please visit here.

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from 663e3a5 to 70928e6 Compare October 20, 2025 19:33
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 20, 2025

/retest

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from 70928e6 to c3dc08a Compare October 21, 2025 14:36
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 21, 2025

/retest

@github-actions
Copy link
Contributor

github-actions bot commented Oct 21, 2025

The image is available at:

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from c3dc08a to eb02bdd Compare October 21, 2025 18:22
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 21, 2025

/retest

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from eb02bdd to 4a5a596 Compare October 21, 2025 20:02
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 21, 2025

/retest

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from 4a5a596 to a038302 Compare October 22, 2025 14:33
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 22, 2025

/retest

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from a038302 to 783d0b5 Compare October 22, 2025 14:37
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 22, 2025

/retest

@JessicaJHee JessicaJHee marked this pull request as ready for review October 22, 2025 15:37
@openshift-ci openshift-ci bot requested review from gustavolira and psrna October 22, 2025 15:37
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 22, 2025

/retest

7 similar comments
@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 22, 2025

/retest

@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 23, 2025

/retest

@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 23, 2025

/retest

@albarbaro
Copy link
Member

albarbaro commented Oct 24, 2025

/retest

@albarbaro
Copy link
Member

albarbaro commented Oct 24, 2025

/retest

@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 27, 2025

/retest

@JessicaJHee
Copy link
Member Author

JessicaJHee commented Oct 29, 2025

/retest

dzemanov
dzemanov requested changes Oct 31, 2025
View reviewed changes
@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from 783d0b5 to 9a0b070 Compare November 3, 2025 16:14
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

The image is available at:

@JessicaJHee JessicaJHee force-pushed the e2e-rbac-conditional-prio branch from 9a0b070 to 81ed2fe Compare November 5, 2025 14:23
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

The image is available at:

dzemanov
dzemanov approved these changes Nov 6, 2025
View reviewed changes
Copy link
Member

@dzemanov dzemanov left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. thank you!

@dzemanov
Copy link
Member

dzemanov commented Nov 6, 2025

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm label Nov 6, 2025
@openshift-ci
Copy link

openshift-ci bot commented Nov 6, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dzemanov
Once this PR has been reviewed and has the lgtm label, please assign gustavolira for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dzemanov dzemanov dzemanov approved these changes

@gustavolira gustavolira Awaiting requested review from gustavolira

@psrna psrna Awaiting requested review from psrna

@AndrienkoAleksandr AndrienkoAleksandr Awaiting requested review from AndrienkoAleksandr

Assignees

@dzemanov dzemanov

Labels

lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JessicaJHee @albarbaro @dzemanov